Skip to content

Shicheng/validate wrapper#483

Merged
ShiCheng-Lu merged 3 commits intomasterfrom
shicheng/validate-wrapper
Dec 1, 2025
Merged

Shicheng/validate wrapper#483
ShiCheng-Lu merged 3 commits intomasterfrom
shicheng/validate-wrapper

Conversation

@ShiCheng-Lu
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 1, 2025 21:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Gradle wrapper validation approach and bumps the SDK version. The changes modify the release workflow to use the newer gradle/actions/wrapper-validation@v3 action instead of the deprecated gradle/wrapper-validation-action, and add pull request triggers to test the validation. However, the build/publish steps have been commented out, which prevents the workflow from functioning as intended.

Key changes:

  • Updated Gradle wrapper validation action to gradle/actions/wrapper-validation@v3
  • Bumped SDK version from 3.23.5 to 3.24.0
  • Added pull request triggers and commented out the Maven publication steps

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
sdk/build.gradle Version bump from 3.23.5 to 3.24.0
.github/workflows/radar-release-actions.yml Updated wrapper validation action, added PR triggers, and commented out build/publish steps

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ShiCheng-Lu ShiCheng-Lu merged commit a48ce16 into master Dec 1, 2025
1 check passed
@ShiCheng-Lu ShiCheng-Lu deleted the shicheng/validate-wrapper branch December 1, 2025 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants